Hpa #3492
Hpa #3492
Conversation
add: resources for nginx
|
Hi @nowjean! Welcome to the project! 🎉 Thanks for opening this pull request! |
|
✅ All required contributors have signed the F5 CLA for this PR. Thank you! |
|
I have hereby read the F5 CLA and agree to its terms |
|
Thank you for you contribution to the project. Please run |
| autoscaling: | ||
| enabled: false | ||
| annotations: {} | ||
| minReplicas: 1 | ||
| maxReplicas: 11 | ||
| targetCPUUtilizationPercentage: 50 | ||
| targetMemoryUtilizationPercentage: 50 | ||
| behavior: {} | ||
| # scaleDown: | ||
| # stabilizationWindowSeconds: 300 | ||
| # policies: | ||
| # - type: Pods | ||
| # value: 1 | ||
| # periodSeconds: 180 | ||
| # scaleUp: | ||
| # stabilizationWindowSeconds: 300 | ||
| # policies: | ||
| # - type: Pods | ||
| # value: 2 | ||
| # periodSeconds: 60 | ||
| autoscalingTemplate: [] | ||
| # Custom or additional autoscaling metrics | ||
| # ref: https://kubernetes.io/docs/tasks/run-application/horizontal-pod-autoscale/#support-for-custom-metrics | ||
| # - type: Pods | ||
| # pods: | ||
| # metric: | ||
| # name: nginx_gateway_fabric_nginx_process_requests_total | ||
| # target: | ||
| # type: AverageValue | ||
| # averageValue: 10000m | ||
|
|
There was a problem hiding this comment.
nit: add one-line descriptions for these fields, similar to how it's done for the others, to improve readability for users.
github-actions
bot
added
the
documentation
I’ve completed 'make generate-all'. Could you please review my PR? |
nowjean
force-pushed
the
hpa
branch
2 times, most recently
from
172c009 to
d081d68
Compare
Proposed changes
Write a clear and concise description that helps reviewers understand the purpose and impact of your changes. Use the
following format:
Problem: I want NGF to work with a HorizontalPodAutoscaler
Solution: Add HPA for deployement
Testing: Describe any testing that you did.
I've deployed my AKS cluster and checked hpa working correctly.
Closes #3447
Checklist
Before creating a PR, run through this checklist and mark each as complete.
Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.